Skip to content

Implement SupportedTokenTypes() on all credential structs#255

Merged
TomK merged 3 commits intomasterfrom
implement-supported-token-types
Mar 4, 2026
Merged

Implement SupportedTokenTypes() on all credential structs#255
TomK merged 3 commits intomasterfrom
implement-supported-token-types

Conversation

@TomK
Copy link
Copy Markdown
Contributor

@TomK TomK commented Mar 4, 2026

Summary

  • Adds SupportedTokenTypes() method to all 41 credential structs satisfying the Credentials interface
  • Token sources aligned with v2 connector GetDefinition responses and psp-proxy implementations
  • Non-payment connectors (fraud, 3DS, account updaters, tokenizers, recovery agents) return nil

Token Source Mapping

Token Sources Connectors
PAN, Connector, NetworkToken Stripe, Adyen, Braintree, Worldpay, Checkout, PaySafe, Sandbox, Sandbanx, PayPalCompletePayments
PAN, NetworkToken BlueSnap, Nuvei, Authorize, Clearhaus, CWAMS, EPX, Woodforest, Qualpay
PAN, Connector Bottomline, PayPalExpressCheckout, PayPalWebsitePaymentsPro
PAN Yapstone, InovioPay, Vindicia, TrustPayments
nil all non-payment connectors

Test Plan

  • go build ./... passes
  • go test ./... passes

🤖 Generated with Claude Code

TomK and others added 3 commits March 4, 2026 17:19
Enables Assemble to determine at placement time which token sources
each payment gateway can process, avoiding expensive gRPC calls to
v2 connector services. Token type values aligned with v2 connector
GetDefinition and psp-proxy implementations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
TrustPayments uses pcib.HvtPan() - should return PAN, not nil.
Woodforest v2 connector defines {TS_NETWORK_TOKEN, TS_PAN}.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@TomK TomK merged commit ab363ca into master Mar 4, 2026
1 check passed
@TomK TomK deleted the implement-supported-token-types branch March 4, 2026 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants